-
-
Notifications
You must be signed in to change notification settings - Fork 431
ATL-1206, 1195, 1193, 1207, 1215, 786: Various fixes and improvements for ATL Sprint 20 #303
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ATL-1206, 1195, 1193, 1207, 1215, 786: Various fixes and improvements for ATL Sprint 20 #303
Conversation
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
We decided to run the gRPC equivalent of |
Show all examples: - when no board is selected, - when the core is not installed for the selected board. Otherwise, show examples for the currently selected board only. Only get libraries from the cores when the FQBN is defined. Otherwise, we retrieve user installed libraries only. Signed-off-by: Akos Kitta <[email protected]>
Like for the examples, we show all libraries if no board is selected, or the platform for the selected board is not installed. Otherwise, we show the libs for the current board. Signed-off-by: Akos Kitta <[email protected]>
Signed-off-by: Akos Kitta <[email protected]>
When running the handler for the `Sketchbook` menu, do not clone the sketch. Closes #297 Signed-off-by: Akos Kitta <[email protected]>
Signed-off-by: Akos Kitta <[email protected]>
9222b1f
to
0b3f899
Compare
hi @kittaakos
right now I cannot have the "no board selected" situation because of ATL-1206, so I erased the preference folder
because of the above I cannot test this one.
|
If you want to "unset" the selected board, you have to open the DevTools, write screencast.2021-04-08.17-26-32.mp4 |
A sketch file cannot contain space. Your macOS-based copy/paste will result in an invalid sketch file name. So the IDE does not open that file. This behaves the same with the Java IDE. I do not think this is a bug. |
Closes #95. Signed-off-by: Akos Kitta <[email protected]>
86e46c4
to
3fc1ffc
Compare
8e16c7e
to
cdd3315
Compare
I found this in the logs:
It looks suspicious. unknown package arduino; there is no package |
This happens when calling |
I thought the same, but no. |
|
Everything seems to work, but I'm still able to create a new Tab with a space in the name. |
FYI: We have an issue with the CLI, I believe. It's super easy to reproduce, wipe |
Signed-off-by: Akos Kitta <[email protected]>
4531fd3
to
9bf2219
Compare
This PR is blocked by eclipse-theia/theia#9349. |
Signed-off-by: Akos Kitta <[email protected]>
Signed-off-by: Akos Kitta <[email protected]>
So that the IDE correctly loads the VS Code extensions. Signed-off-by: Akos Kitta <[email protected]>
9bf2219
to
90d2ba8
Compare
I have fixed the outstanding issues. It's ready for review. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested and approved
thank you
Thank you for the review. I am merging the changes to the |
Sketchbook
handler.0.18.1
CLI.lib list
andboard details
throw incorrect error whendirectories.data
folder does not exist at daemon startup arduino-cli#1262.